ci(actions): GitHub Actions RTL/Web/WASM pipelines + Vercel deploy config (Phase 9) - #13
Merged
Conversation
Adds .github/workflows/rtl.yml, triggered on push/PR to rtl/** or sim/**. Ubuntu 24.04 apt ships Verilator 5.020, but cocotb 2.x requires >= 5.036. Build Verilator v5.036 from source on first run and cache the install prefix by version tag so subsequent runs skip the ~5 min build step. Steps: - Lint RTL with `verilator --lint-only -Wall rtl/*.sv` - Run `pytest sim/golden.py -q` (numpy golden reference) - cocotb test_pe, test_systolic_array, test_top via make (WAVES=0 for speed) Removes .github/workflows/.gitkeep placeholder now that real files exist.
Adds .github/workflows/web.yml, triggered on push/PR to web/**. Uses pnpm/action-setup@v4 (pnpm 11) + actions/setup-node@v4 (Node 22) with `cache: pnpm` pointing at web/pnpm-lock.yaml to avoid full reinstalls on unchanged dependencies. Runs in order: pnpm install --frozen-lockfile, pnpm lint, pnpm typecheck, pnpm build. All three must pass; build failure surfaces SSR/WASM import errors that wouldn't otherwise be caught locally.
Adds .github/workflows/wasm.yml, triggered on push/PR to rtl/** or wasm/**. Installs Verilator 5.020 from apt (sufficient for the WASM build path — cocotb's version gate only applies to the simulation Makefile, not the em++ compilation step) and Emscripten via mymindstorm/setup-emsdk@v14. Runs wasm/build.sh and asserts both web/public/tiny_tpu.mjs and web/public/tiny_tpu.wasm are produced. Does not commit artifacts from CI; they must be committed manually after local rebuilds.
Adds a second badge row beneath the tech-stack badges with live status links for the three new GitHub Actions workflows: RTL CI, Web CI, and WASM Build. Badges link directly to the workflow run list so viewers can inspect recent results without leaving the README.
Vercel may serve .wasm files as text/html without an explicit header rule, which causes the browser to reject the module with a MIME type error and shows a blank visualizer. The headers block pins Content-Type to application/wasm for tiny_tpu.wasm and application/javascript for tiny_tpu.mjs, ensuring correct browser parsing in production. Also sets Cache-Control: immutable on both artifacts — they are content- addressed by the WASM build process so long-lived caching is safe. vercel.json is placed inside web/ (the configured Vercel root directory).
Two issues in the OG image generation script broke `pnpm lint`: 1. Unused variable: `fill` was computed in peCell() but the SVG template later switched to inline hardcoded hex values. Removed the dead declaration so no-unused-vars is satisfied. 2. Undeclared globals: `console` and `Buffer` are Node.js globals, but the ESLint config only injected browser + es2022 globals for .ts/.tsx files. Added a separate config block for scripts/**/*.mjs with globals.node so these built-ins are recognised without any eslint-disable comments. `pnpm lint`, `pnpm typecheck`, and `pnpm build` all pass after this fix.
deaneeth
changed the title
cocotb does not detect when TOPLEVEL changes between sequential make runs sharing the same sim_build/ directory. The PE test compiled sim_build/Vtop for module `pe`; the systolic array test then reused that binary and crashed at runtime with "root handle systolic_array != pe". Fix: pass SIM_BUILD=sim_build/<suite> to each make invocation so every test suite compiles into its own isolated directory.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
web/vercel.jsonwith explicitContent-Type: application/wasmand long-livedCache-Control: immutableheaders — without this Vercel serves the binary astext/htmland the browser rejects the WASM modulegen-og-image.mjs(unusedfillvariable +console/BufferNode.js globals not configured) that would have failed Web CI on the very first runType of Change
ci— GitHub Actions changechore— repo maintenance, CI, toolingfix— bug fixdocs— documentation onlyWhat Was Changed and Why
Key decisions
cocotb2.x (installed in the venv) requires Verilator >= 5.036; Ubuntu 24.04 apt ships only 5.020. The build is pinned to tagv5.036and cached by that tag so only the first CI run pays the ~5 min compile cost.verilator --cc→em++) has no cocotb version gate, so the faster apt install is correct and sufficient there.vercel.jsonlives inweb/: Vercel root directory is configured asweb/, so it readsvercel.jsonfrom there. The explicit MIME header for.wasmis required — without it Vercel serves the binary astext/htmland the browser rejects the WASM module with a type error.scripts/: Addedglobals.nodeonly forscripts/**/*.mjs— not globally — to preserve browser-only type safety for all application source files.RTL Changes
N/A — no RTL source files were modified.
Frontend Changes
pnpm lint— cleanpnpm typecheck— clean (0 errors, 0 warnings across 32 files)pnpm build— clean, 7 pages built successfullyTesting
Branching Checklist
devdevci:,docs:,chore:,fix:)Reviewer Notes
test_pe,test_systolic_array,test_top) can only run in CI until the local toolchain is upgraded by building Verilator from source. The golden model tests (pytest sim/golden.py) still run fine locally.web/public/tiny_tpu.mjsandweb/public/tiny_tpu.wasmexist. These were committed in Phase 5 and are present — the workflow will pass.